Skip to content

dd#82

Merged
Juhye0k merged 1 commit intodevfrom
sql
Sep 27, 2025
Merged

dd#82
Juhye0k merged 1 commit intodevfrom
sql

Conversation

@Juhye0k
Copy link
Contributor

@Juhye0k Juhye0k commented Sep 27, 2025

What is this PR?🔍

추가하려는 기능에 대해 간결하게 설명해주세요(작성 시 삭제)

Changes💻

ScreenShot📷

Summary by CodeRabbit

  • 유지보수(Chores)
    • 요청 서명 검증 단계의 디버그 로깅을 대폭 강화하여 문제 진단과 추적을 용이하게 했습니다.
    • 요청 본문과 계산된 값의 비교 결과를 명확히 기록하고, 불일치 시 오류 로그를 추가해 원인 파악 시간을 단축합니다.
    • 동작 흐름과 공개 API에는 변화가 없으며, 사용자 기능에는 영향이 없습니다. 성능 영향은 미미합니다.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Sep 27, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

CallbackService.verifySignatureOrThrow에 HMAC 검증 과정의 디버그 로그가 대폭 추가되었으며, 요청 JSON/페이로드/수신 서명/계산된 기대 서명을 단계별로 기록합니다. 제어 흐름, 예외 처리, 공개 시그니처 변화는 없습니다.

Changes

Cohort / File(s) Summary
CallbackService HMAC 디버그 로깅 추가
src/main/java/avengers/lion/mission/service/CallbackService.java
HMAC 검증 전후로 JobId, 요청 JSON, 페이로드, 수신 서명, 계산된 기대 서명을 로깅. 서명 불일치 시 에러 로그 추가. 로직/흐름 변화 없음.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

작은 토끼, 로그를 톡톡 남겨두네
해시의 길을 한 줄씩 밝혀보네
받은 서명, 만든 서명 나란히 두고
수풀 속 버그도 더는 못 숨고
깡총, 깡총—디버그 끝! 🐇✨

✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch sql

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 890322a and 3b84bb3.

📒 Files selected for processing (1)
  • src/main/java/avengers/lion/mission/service/CallbackService.java (2 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Juhye0k Juhye0k merged commit ff84153 into dev Sep 27, 2025
1 of 2 checks passed
@seo-bo
Copy link

seo-bo commented Sep 29, 2025

403589-backgroundImage1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants